Skip to content

#48 [Feat] 전역 API 로깅 및 JwtFilter 예외 응답 로직 구현#49

Merged
si-zero merged 11 commits intodevfrom
feat/#48
Mar 24, 2026
Merged

#48 [Feat] 전역 API 로깅 및 JwtFilter 예외 응답 로직 구현#49
si-zero merged 11 commits intodevfrom
feat/#48

Conversation

@si-zero
Copy link
Contributor

@si-zero si-zero commented Mar 24, 2026

#️⃣ 연관된 이슈

📝 작업 내용

✨ Feat

내용 파일
로그아웃 및 회원탈퇴 DTO 구현 LogoutResponse.java, WithdrawnResponse.java
전역 API 호출 로깅 기능 추가 ApiLoggingAspect.java

♻️ Refactor

내용 파일
인증 컨트롤러 DTO 추가에 따라 리팩토링 AuthService.java
서블릿 컨테이너 도달 전 에러 던지는 부분 리팩토링 JwtFilter.java

🧪 Test

내용 파일
로그인 테스트 추가 AuthServiceTest.java

📌 공유 사항

없음

✅ 체크리스트

  • Reviewer에 팀원들을 선택했나요?
  • Assignees에 본인을 선택했나요?
  • 컨벤션에 맞는 Type을 선택했나요?
  • Development에 이슈를 연동했나요?
  • Merge 하려는 브랜치가 올바르게 설정되어 있나요?
  • 컨벤션을 지키고 있나요?
  • 로컬에서 실행했을 때 에러가 발생하지 않나요?
  • 팀원들에게 PR 링크 공유를 했나요?

📸 스크린샷

API 로깅

스크린샷 2026-03-24 오전 10 31 39

💬 리뷰 요구사항

  1. 현재 명세서랑 실제 API 응답 데이터 형식이 camelCase 인지 snake_case 통일이 안되어 있는 것 같은데 어떻게 하면 좋을지 여쭤보고 싶습니다.

@si-zero si-zero self-assigned this Mar 24, 2026
@si-zero si-zero added the ✨ Feat (기능 추가) 새로운 기능을 추가할 때 사용합니다. label Mar 24, 2026
@si-zero si-zero linked an issue Mar 24, 2026 that may be closed by this pull request
3 tasks
Copy link
Member

@jucheonsu jucheonsu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

코드 구현하시느라 고생 많으셨습니다! 리뷰 요구사항에 남겨주신 부분은 snake_case로 관리하는 것이 좋을 것 같다고 생각하는데 그렇게 되면 Response DTO에 작성해주신 어노테이션을 모두 적용하는 작업을 진행해야할 것 같습니다!

@si-zero si-zero merged commit 0b32654 into dev Mar 24, 2026
@si-zero si-zero deleted the feat/#48 branch March 24, 2026 13:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

✨ Feat (기능 추가) 새로운 기능을 추가할 때 사용합니다.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

✨ Feat: 전역 API 로깅(AOP) 및 JwtFilter 예외 응답 로직 구현

2 participants